Skip to content

fix: 이슈 #404, #403, #387, #370, #366 수정#412

Merged
manNomi merged 5 commits intomainfrom
fix/issues-404-403-387-370-366
Feb 14, 2026
Merged

fix: 이슈 #404, #403, #387, #370, #366 수정#412
manNomi merged 5 commits intomainfrom
fix/issues-404-403-387-370-366

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Feb 2, 2026

요약

5개 이슈 수정: 로그인 리다이렉션, 멘토 알림 API, 회원가입 로직, 어드민 role, 학교 정보 로딩

수정 내역

#404 - 로그인 후 페이지 리다이렉션 타이밍 이슈

  • Zustand persist middleware가 localStorage에 토큰을 저장하기 전에 라우터가 리다이렉트되는 문제 해결
  • setTimeout(100ms)으로 토큰 저장 시간 보장 후 리다이렉트

#403 - 멘토 알림 API 불필요한 호출 방지

  • 인증 상태 초기화 전에 멘토 알림 API가 호출되는 문제 해결
  • isLoading 대신 isInitialized를 사용하여 정확한 인증 상태 확인

#387 - 회원가입 및 OAuth 로그인 후 토큰 저장 타이밍 이슈

  • 이메일/카카오/애플 회원가입 및 로그인 완료 후 리다이렉트 시 토큰 저장 보장
  • #404와 동일한 타이밍 이슈 해결 방식 적용

#370 - 어드민 role 처리 개선

  • MentorClient에서 isInitialized 사용으로 정확한 인증 상태 확인
  • ModifyContent에서 어드민 role 표시 및 수학학교 정보 표시 추가
  • 어드민 사용자 타입을 "어드민"으로 올바르게 표시

#366 - 학교 정보 로딩 실패 시 에러 로깅 추가

  • 서버 사이드 대학교 정보 조회 API 실패 시 콘솔 에러 로깅 추가
  • 디버깅 및 문제 해결을 위한 가시성 개선
  • getUniversityDetail, getUniversitiesByText, getSearchUniversitiesByFilter 등 모든 서버 API에 에러 핸들링 추가

검증

  • ✅ TypeCheck: 기존 에러만 존재 (본 PR과 무관)
  • ✅ Build: 성공
  • ✅ 에러 로깅 동작 확인

관련 이슈

Closes #404
Closes #403
Closes #387
Closes #370
Closes #366

@coderabbitai
Copy link

coderabbitai bot commented Feb 2, 2026 •
edited
Loading

Caution

Review failed

The pull request is closed.

Walkthrough

이 PR은 인증 흐름과 API 에러 처리, 그리고 사용자 역할 관련 로직을 개선하는 통합 업데이트입니다. 주요 변경 사항은 다음과 같습니다:

  1. 네비게이션 타이밍 개선 — Apple, Kakao, 이메일 로그인에서 즉시 리다이렉트 대신 100ms 지연된 router.push를 사용하도록 변경하여 Zustand 상태 지속성이 완료될 시간을 제공합니다.

  2. 대학 조회 API 에러 처리 강화 — 추천 대학, 필터 검색, 텍스트 검색, 상세 조회 함수에서 명시적 에러 로깅 및 폴백 처리(빈 배열 또는 undefined 반환)를 추가했습니다.

  3. 역할 기반 로직 확장 — 프로필 수정 페이지에서 ADMIN 역할도 기본 대학 설정과 타입 표시 로직에 포함시켰습니다.

  4. 초기화 상태 검증 변경 — 멘토 적용 카운트 컴포넌트에서 isLoading 대신 isInitialized 조건을 사용하도록 변경했습니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Suggested reviewers

  • wibaek
  • enunsnv
  • khwww
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/issues-404-403-387-370-366

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@manNomi
fix: 로그인 후 페이지 리다이렉션 타이밍 이슈 수정
1f5582f 
Zustand persist middleware가 localStorage에 토큰을 저장하기 전에
라우터가 리다이렉트되는 타이밍 이슈 해결

Closes #404
@manNomi
fix: 멘토 알림 API 불필요한 호출 방지
d642a75 
인증 상태 초기화 전에 멘토 알림 API가 호출되는 문제 해결
isLoading 대신 isInitialized를 사용하여 정확한 인증 상태 확인

Closes #403
@manNomi
fix: 회원가입 및 OAuth 로그인 후 토큰 저장 타이밍 이슈 수정
5984988 
이메일/카카오/애플 회원가입 및 로그인 완료 후 리다이렉트 시
Zustand persist middleware의 토큰 저장을 보장하도록 개선

Closes #387
@manNomi
fix: 어드민 role 처리 개선
4d3d590 
- MentorClient에서 isInitialized 사용으로 정확한 인증 상태 확인
- ModifyContent에서 어드민 role 표시 및 수학학교 정보 표시 추가

Closes #370
@manNomi
fix: 학교 정보 로딩 실패 시 에러 로깅 추가
9c7d49b 
서버 사이드 대학교 정보 조회 API 실패 시 콘솔 에러 로깅 추가로
디버깅 및 문제 해결을 위한 가시성 개선

Closes #366
@manNomi manNomi force-pushed the fix/issues-404-403-387-370-366 branch from 3dc0979 to 9c7d49b Compare February 14, 2026 09:35
@vercel
Copy link

vercel bot commented Feb 14, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
solid-connect-web-admin Error Error Feb 14, 2026 9:50am
solid-connection-web Ready Ready Preview, Comment Feb 14, 2026 9:50am

@github-actions github-actions bot added the web label Feb 14, 2026
@manNomi manNomi merged commit 58ee078 into main Feb 14, 2026
6 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wibaek wibaek Awaiting requested review from wibaek wibaek is a code owner

@enunsnv enunsnv Awaiting requested review from enunsnv enunsnv is a code owner

@khwww khwww Awaiting requested review from khwww khwww is a code owner

Assignees

No one assigned

Labels

web

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

로그인 이후 페이지 미이동 멘토 알림 api 회원 가입 로직 수정 어드민 role 처리 학교 정보 못불러오는 버그

1 participant

@manNomi